Skip to content

Conversation

@nghielme
Copy link

This PR contain a set of bug fix encountered during the ingestion of a UNet like model.

Type of change

Different kind of bug fixes.

Note: Please delete options that are not relevant.

  • Bug fix (non-breaking change that fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Tests

Missing at the moment but they will be added in a second time.

Checklist

  • I have read the guidelines for contributing.
  • My changes generate no new warnings.
  • I have installed and run pre-commit on the files I edited or added.
  • I have added tests that prove my fix is effective or that my feature works.

"""
rv = []
for inp in node.input:
if inp == '':
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When is this used? When are you asking for the input node of a node with no inputs?

Copy link
Author

@nghielme nghielme Sep 20, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It happens with Resize node, https://onnx.ai/onnx/operators/onnx__Resize.html
I agree it is not super clean but I didn't figure out another solution. The handling of RoI field should also be considered at QONNX level

jmitrevs pushed a commit that referenced this pull request Feb 12, 2025
Fix pytorch simple RNN for oneAPI; add initial state version for Quartus and oneAPI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants